Skip to content

Conversation

@csells
Copy link
Contributor

@csells csells commented Nov 7, 2025

Pre-launch Checklist

  • I signed the CLA.

@csells csells requested a review from redbrogdon November 7, 2025 22:04
Copy link
Contributor

@redbrogdon redbrogdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple quick nits. Do you mind also adding a license header to each code file (e.g. https://github.com/flutter/demos/blob/main/compass_25/lib/model/activity.dart).

There's a single LICENSE file for the repo, so you don't need to add one here.


### Prerequisites

- A Firebase project.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this link to Firebase's docs for creating one? Also, add a note that the project needs Generative AI to be turned on (and possibly link to that doc as well).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated. ptal.

@@ -0,0 +1,4 @@
flutter clean
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd leave this file out.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

@csells
Copy link
Contributor Author

csells commented Nov 10, 2025

I believe all of the feedback has been addressed, but please double-check me.

@redbrogdon redbrogdon merged commit 4f5bc1d into flutter:main Nov 11, 2025
1 check passed
@redbrogdon
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants